Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[FEATURE] JSON 比对 #1946 #2267

Open
wants to merge 22 commits into
base: main
Choose a base branch
from
Open

[FEATURE] JSON 比对 #1946 #2267

wants to merge 22 commits into from
+1,384 −0

Conversation

yanchangyou
Copy link

@yanchangyou yanchangyou commented Feb 25, 2024 •
edited
Loading

What this PR does / why we need it?

[FEATURE] JSON 比对 #1946

Summary of your change

add compare util

Please indicate you've done the following:

  • Made sure tests are passing and test coverage is added if needed.
  • Made sure commit message follow the rule of Conventional Commits specification.
  • Considered the docs impact and opened a new docs issue or PR with docs changes if needed.

@imflyfish
add json compare util
75c3cd1
@imflyfish
add json compare util
e8902a6 
[FEATURE] JSON 比对 alibaba#1946
@imflyfish
Merge remote-tracking branch 'origin/main'
7bfb932 
# Conflicts:
#	core/src/test/java/com/alibaba/fastjson2/util/CompareUtilsTest.java
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@codecov-commenter
Copy link

codecov-commenter commented Feb 29, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 98.07692% with 3 lines in your changes missing coverage. Please review.

Project coverage is 77.47%. Comparing base (a0b63fa) to head (3f7166f).
Report is 492 commits behind head on main.

Files with missing lines Patch % Lines
.../java/com/alibaba/fastjson2/util/CompareUtils.java 98.07% 0 Missing and 3 partials ⚠️
Additional details and impacted files 
@@             Coverage Diff              @@
##               main    #2267      +/-   ##
============================================
+ Coverage     77.36%   77.47%   +0.10%     
- Complexity    22613    22784     +171     
============================================
  Files           634      635       +1     
  Lines         84663    85377     +714     
  Branches      19231    19354     +123     
============================================
+ Hits          65496    66142     +646     
- Misses        11171    11232      +61     
- Partials       7996     8003       +7

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@yanchangyou yanchangyou mentioned this pull request Mar 13, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@yanchangyou @CLAassistant @codecov-commenter @imflyfish